Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

test: Add more tests / fix removal of coverage dir #431

Merged
merged 5 commits into from
Dec 2, 2024
Merged

Conversation

matz3
Copy link
Member

@matz3 matz3 commented Nov 29, 2024

No description provided.

@matz3 matz3 requested a review from a team November 29, 2024 09:37
@matz3 matz3 force-pushed the test-improvements branch from 76ba527 to 8a3099f Compare November 29, 2024 09:49
d3xter666
d3xter666 previously approved these changes Dec 2, 2024
Copy link
Contributor

@d3xter666 d3xter666 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@d3xter666 d3xter666 requested a review from a team December 2, 2024 07:42
RandomByte
RandomByte previously approved these changes Dec 2, 2024
@matz3 matz3 dismissed stale reviews from RandomByte and d3xter666 via c7777ca December 2, 2024 09:44
@matz3 matz3 force-pushed the test-improvements branch from 8a3099f to c7777ca Compare December 2, 2024 09:44
@matz3 matz3 requested review from RandomByte and d3xter666 December 2, 2024 09:44
@matz3 matz3 force-pushed the test-improvements branch from c7777ca to 909acc6 Compare December 2, 2024 09:45
@matz3
Copy link
Member Author

matz3 commented Dec 2, 2024

Paths on windows should be fixed now. In our results we always use posix-style.

@matz3 matz3 force-pushed the test-improvements branch 4 times, most recently from ee6aeb5 to 689c753 Compare December 2, 2024 12:32
@matz3 matz3 force-pushed the test-improvements branch from 689c753 to d8ff46b Compare December 2, 2024 12:42
@matz3 matz3 merged commit d50b8fe into main Dec 2, 2024
13 checks passed
@matz3 matz3 deleted the test-improvements branch December 2, 2024 13:38
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants